Skip to content

Conversation

@dnhatn
Copy link
Member

@dnhatn dnhatn commented Feb 3, 2025

The upper bound of randomVersionBetween is inclusive; therefore, for
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes #117937

@dnhatn dnhatn added >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL v8.17.2 v8.16.4 v8.18.1 v8.19.0 v9.0.1 v9.1.0 backport auto-backport Automatically create backport pull requests when merged and removed backport labels Feb 3, 2025
@dnhatn dnhatn marked this pull request as ready for review February 3, 2025 20:55
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Feb 3, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@dnhatn dnhatn force-pushed the fix-cluster-requests branch from 03b6697 to 179b558 Compare February 3, 2025 22:20
@elasticsearchmachine elasticsearchmachine merged commit c717e78 into elastic:main Feb 3, 2025
17 checks passed
@dnhatn dnhatn deleted the fix-cluster-requests branch February 3, 2025 23:30
dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Feb 3, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for 
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes elastic#117937
elasticsearchmachine pushed a commit that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for 
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes #117937
dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes elastic#117937
dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes elastic#117937
dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes elastic#117937
elasticsearchmachine pushed a commit that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes #117937
elasticsearchmachine pushed a commit that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes #117937
@elastic elastic deleted a comment from elasticsearchmachine Feb 4, 2025
dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes elastic#117937
dnhatn added a commit that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes #117937
elasticsearchmachine pushed a commit that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes #117937
fzowl pushed a commit to voyage-ai/elasticsearch that referenced this pull request Feb 4, 2025
The upper bound of randomVersionBetween is inclusive; therefore, for 
testing the fallback version of the request, we need to use the version
preceding 8.16.0 rather than 8.16.0 itself.

Closes elastic#117937
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.16.4 v8.17.2 v8.18.1 v8.19.0 v9.0.1 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] ClusterRequestTests testFallbackIndicesOptions failing

2 participants